-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Adding MSETEX command support. #3823
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds support for the Redis MSETEX command, which sets multiple key-value pairs with optional expiration settings. The implementation introduces tests and cluster support for this new command.
- Adds the
DataPersistOptionsenum (NX/XX) to control conditional setting behavior - Implements the
msetexmethod inredis/commands/core.pywith support for various expiration options (ex, px, exat, pxat, keepttl) - Adds comprehensive test coverage for both sync and async clients
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 10 comments.
| File | Description |
|---|---|
| redis/commands/core.py | Adds DataPersistOptions enum and implements the msetex method with expiration and conditional setting options |
| redis/cluster.py | Registers "MSETEX" as a cluster-compatible command |
| tests/test_commands.py | Adds comprehensive test cases for msetex with various expiration options and conditional settings |
| tests/test_asyncio/test_commands.py | Adds async versions of msetex test cases |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
2492b5f to
81abba2
Compare
…and pipelines in clusters
81abba2 to
a24b9c2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 7 out of 7 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Pull Request check-list
Please make sure to review and check all of these items:
NOTE: these things are not required to open a PR and can be done
afterwards / while the PR is open.
Description of change
Please provide a description of the change here.